Skip to content

Conversation

@OctagonalStar
Copy link
Contributor

  • Have you followed the contributing guidelines?
  • Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


what your changes do

Fix editorial inconsistencies or inaccuracies:

  • Replace some english symbol in Chinese sentences. 5730b11
  • Fix some brackets miss match. ca4eb9d
  • Removed an empty row (because MD012/no-multiple-blanks: Multiple consecutive blank lines) 5730b11

why they add value to the Guides

  • It can make Simplified Chinese native speakers easier to understand the meaning of the guides.
  • It can make the copied text more accurate.

The original zh-hans document used a mix of Chinese brackets("(" or ")") and English brackets incorrectly.

Signed-off-by: OctagonalStar <[email protected]>
Use the Chinese punctuation mark in zh-hans document.

Signed-off-by: OctagonalStar <[email protected]>
@OctagonalStar
Copy link
Contributor Author

It looks the chinese quote symbol is not allowed. Should I reset the commit 5730b11 ?

@OctagonalStar
Copy link
Contributor Author

I have solve the warning issue in the Github Action log in the first run.
The actual failures are due to the external link checker.
It got HTTP 403 responses from medium.com.
But I didn't change any of the links. It's strange.

@jmeridth
Copy link
Member

@OctagonalStar I fixed the 403 issue and merged latest main branch.

hblwslyan-commits

This comment was marked as spam.

@OctagonalStar
Copy link
Contributor Author

Thank you @jmeridth for fixing the 403 issue.

I think I have completed all the changes. Anyone review the changes?

Copy link
Member

@jmeridth jmeridth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few questions. Are the extra spaces before and after intentional? and a quotation placement.

@jmeridth jmeridth added this pull request to the merge queue Dec 4, 2025
Merged via the queue into github:main with commit 4fe42a0 Dec 4, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants